-
-
Notifications
You must be signed in to change notification settings - Fork 158
fix: schedule write action for edit-special. #758
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
base: master
Are you sure you want to change the base?
fix: schedule write action for edit-special. #758
Conversation
I see some tests are failed. However, I have no idea why scheduling this action would fail so many tests? In my use case, it works well. I think this might because |
Can you give me the steps to reproduce the issue? I can't reproduce it so can't really help. |
This is a problem that's are really hard to reproduce. The same config can yield the "E315: ml_get: invalid lnum: 1" on macos, but not on linux. And I cannot create a minimal config either. May need to spend more time on this to find what's triggering the issue. Another interesting thing is that:
So this might point to a C source code error. |
I was encountering this error on task creation until I set It was not until I set these two variables the error went away. Just thought I should comment here if someone else finds this error, too! |
I don’ t use noice. Add some context to see if I can help, but the sad thing is that I cannot reproduce the problem… I encounter this problem when trying to use If I am editing another code block (e.g. not SQL), then I do not encounter that error. And if I don’t use efm with SQL, the problem disapperas too. There are too many variables to control and I cannot reproduce. |
The action of writing to the original buffer need to be schedule wrapped. otherwise, on the first time writing the change to the org buffer, it will issue an
"E315: ml_get: invalid lnum: 1"
error. I guess this might be some race conditions? Anyway schedule this call makes this error disappear.